Repository navigation
fix(python): accurate LLM hint and exit 124 on Monty time limit - #2833
Merged
Merged
Conversation
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
bashkit | 8a6974c | Commit Preview URL Branch Preview URL |
Nov 05 2026, 04:38 AM |
Review summary (head
|
chaliy
force-pushed
the
claude/project-thread-pxlkst-python-hint
branch
from
November 5, 2026 03:53
6292182 to
35cde4e
Compare
2 tasks done
The hint said "No classes" after Monty 0.0.19 added simple classes and never mentioned that `re` is disabled. Rewrite it and pin it with a test that imports every module it names and checks the class/match claims. A Monty time-limit hit returned exit 1 and a TimeoutError traceback, so callers had to parse stderr to tell a limit from a script error. Report it like the CPython runtime: exit 124, 'python3: execution timed out'.
A script could raise TimeoutError('time limit exceeded: ...') and be
reported as a limit hit (exit 124). Record in the tracker when its own
time check fires and key the exit-124 path on that.
chaliy
force-pushed
the
claude/project-thread-pxlkst-python-hint
branch
from
November 5, 2026 04:37
e166090 to
8a6974c
Compare
2 tasks done
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requested by Михайло · project thread
What changed
python3LLM hint describes what Monty actually runs: the real stdlib list (json, datetime, unicodedata were missing),redisabled, simple classes only (no inheritance), nomatch, output viaprint(). A new test imports every module the hint names and checks the class/match/reclaims, so the next Monty upgrade that changes them fails the build instead of silently misleading agents.python3: execution timed out after Ns, keeping earlier output, the same as the CPython runtime and coreutilstimeout. A script's ownTimeoutErroris still an ordinary exit-1 exception.Why
Found reviewing a downstream integration (harnx): its docs had to warn users that bashkit's "no classes" hint is wrong, and its backend greps stderr for
TimeoutError: time limit exceeded:to tell a timeout from a script error.Before / After
Before:
After: stdout
1\nrc=124, stderrpython3: execution timed out after 0.2s(timeout_limit_exits_124). Hint before:... No HTTP/network. No classes. No third-party imports.; after: seePython::llm_hint, enforced bytest_llm_hint_matches_runtime.Risk
Checklist
Generated by Claude Code
Open in [code]smith · Need help on this PR? Tag
@codesmith-botwith what you need.